Skip to content

Add EcosystemList component and integrate new ecosystem data#514

Merged
rootsongjc merged 4 commits into
Project-HAMi:masterfrom
rootsongjc:feat/integrations
Jun 29, 2026
Merged

Add EcosystemList component and integrate new ecosystem data#514
rootsongjc merged 4 commits into
Project-HAMi:masterfrom
rootsongjc:feat/integrations

Conversation

@rootsongjc

@rootsongjc rootsongjc commented Jun 28, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

Introduce the EcosystemList component to display various integrations and add Qwen to the ecosystem. Update styles to accommodate the new layout and ensure proper rendering of logos. Reintroduce the ray.svg logo with its original content.

Summary by CodeRabbit

  • New Features

    • Added a new “Works with HAMi” ecosystem section on the homepage.
    • Expanded the ecosystem showcase with additional partners and workloads.
    • Updated the adopters and logo display areas to use a more flexible logo wall layout.

  • UI Improvements

    • Adjusted the ecosystem grid to show more logos across different screen sizes.
    • Improved logo/name rendering for entries with or without image assets.

@rootsongjc
feat: add EcosystemList component and ecosystem data for integration …
e4764c0 
…showcase

Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
@rootsongjc
Add Qwen to ecosystem and update styles for new layout
13abc6f 
- Added Qwen with its logo to the hero workload ecosystem in index.js.
- Updated styles.module.css to accommodate an additional column in the ecosystem grid.
- Added new SVG file for Qwen logo.
- Reintroduced ray.svg with its original content.

Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
@rootsongjc
style: simplify section title formatting in Home component
6840b29 
Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
@hami-robot

hami-robot Bot commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: rootsongjc
Once this PR has been reviewed and has the lgtm label, please assign archlitchi for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hami-robot hami-robot Bot requested review from archlitchi and windsonsea June 28, 2026 11:34
@netlify

netlify Bot commented Jun 28, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for project-hami ready!

Name Link
🔨 Latest commit 445533f
🔍 Latest deploy log https://app.netlify.com/projects/project-hami/deploys/6a42660466badf0008cae671
😎 Deploy Preview https://deploy-preview-514--project-hami.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@hami-robot hami-robot Bot added the size/L label Jun 28, 2026
@coderabbitai

coderabbitai Bot commented Jun 28, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3b03abae-ecc1-4798-a38f-d3c3d5d86f4c

📥 Commits

Reviewing files that changed from the base of the PR and between 6840b29 and 445533f.

📒 Files selected for processing (2)
  • src/components/logoWall.js
  • src/pages/index.js
📝 Walkthrough

Walkthrough

Adds ecosystem.json with six ecosystem entries, refactors LogoWall from a hardcoded AdoptersList to a props-driven component accepting items and imgPrefix, integrates a new "Works with HAMi" ecosystem section and updated Adopters rendering on the homepage, and widens the scheduler ecosystem grid from 4 to 5 columns.

Changes

Ecosystem Section and LogoWall Refactor

Layer / File(s) Summary
Ecosystem data and LogoWall component refactor
src/data/ecosystem.json, src/components/logoWall.js		 Adds six ecosystem objects (KAI Scheduler, Volcano, Kueue, Koordinator, Olares, CozyStack) in ecosystem.json. Refactors LogoWall to accept items and imgPrefix props, replacing the hardcoded adoptersData import and /img/adopters/ path prefix.
Homepage integration and grid layout
src/pages/index.js, src/pages/styles.module.css		 Imports LogoWall, adopters.json, and ecosystem.json. Inserts Qwen and KAI Scheduler into hero ecosystem arrays. Adds a localized "Works with HAMi" section rendering LogoWall for ecosystem data. Switches the Adopters section from AdoptersList to LogoWall. Updates .schedulerEcosystemGrid from 4 to 5 columns at base and tablet breakpoints, and from 2 to 3 columns at phone breakpoint.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Six new friends join the HAMi crew,
Logos and names, in English and 中文 too.
The wall of logos grows wide and tall,
Props-driven now, one component for all.
Hop hop hooray, the grid has five lanes—
More room for friends on the ecosystem plains! 🌱

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is broadly aligned with the PR’s ecosystem data and integration work, though it misnames the shared component change.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@mesutoezdil

mesutoezdil commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

is line-for-line the same as adoptersList.js, only the data import and fallback logo prefix differ. consider passing data and imgPrefix as props to the existing component instead of a new file.

@rootsongjc
feat: replace EcosystemList and AdoptersList with LogoWall component …
445533f 
…for improved integration display

Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
@hami-robot hami-robot Bot added size/M and removed size/L labels Jun 29, 2026
@mesutoezdil

mesutoezdil commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

/lgtm

@hami-robot hami-robot Bot added the lgtm label Jun 29, 2026
@rootsongjc rootsongjc merged commit dcac812 into Project-HAMi:master Jun 29, 2026
11 of 12 checks passed
@rootsongjc rootsongjc deleted the feat/integrations branch June 29, 2026 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@archlitchi archlitchi Awaiting requested review from archlitchi
@windsonsea windsonsea Awaiting requested review from windsonsea

Assignees

@mesutoezdil mesutoezdil

Labels

dco-signoff: yes lgtm size/M

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rootsongjc @mesutoezdil